Skip to content

Dynamic dedicated opening hours page for users#2151

Open
BragonSB wants to merge 6 commits into
masterfrom
brage/web-169-dynamic-dedicated-opening-hours-page-for-users
Open

Dynamic dedicated opening hours page for users#2151
BragonSB wants to merge 6 commits into
masterfrom
brage/web-169-dynamic-dedicated-opening-hours-page-for-users

Conversation

@BragonSB

@BragonSB BragonSB commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

Closes: WEB-169

This page will almost certainly need a redesign before deployment, however the issue specifies to focus on the functionality. I will therefore not work a lot on the page's design.

Also after speaking with the man, the myth, the legend: Anders we decided to drop the button on the Admin-opening-hours-page

EDIT: I got bored and redesigned the page cause it was incredibly ugly. Would love feedback on the new design

EDIT2: New redesign

@BragonSB BragonSB self-assigned this Apr 15, 2026
@BragonSB
BragonSB marked this pull request as ready for review April 16, 2026 17:25
@eilifhl eilifhl changed the title Brage/web 169 dynamic dedicated opening hours page for users Dynamic dedicated opening hours page for users Apr 25, 2026

@eilifhl eilifhl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Selv om det kanskje ikke er helt ideelt behandles 00:00-00:00 i backend som stengt. Burde kanskje være noe måte å vise det?

Vi burde egt kanskje ha et isOpen felt slik som i samf3, men det er jo en ganske stor endring for denne PRen lol
https://github.com/Samfundet/Samfundet/blob/master/app/models/standard_hour.rb

@eilifhl

eilifhl commented Apr 25, 2026

Copy link
Copy Markdown
Contributor

Selv om det kanskje ikke er helt ideelt behandles 00:00-00:00 i backend som stengt. Burde kanskje være noe måte å vise det?

Vi burde egt kanskje ha et isOpen felt slik som i samf3, men det er jo en ganske stor endring for denne PRen lol https://github.com/Samfundet/Samfundet/blob/master/app/models/standard_hour.rb

nvm, kom på at denne pren, #2096, legger til et slikt felt. kan kanskje merges inn hit

@robines

robines commented Jun 23, 2026

Copy link
Copy Markdown
Member

@BragonSB Liker tanken på et nytt design, men akkurat for åpningstider er det viktig med et funksjonelt design som gir et enkelt overblikk uten at må scrolle eller klikke noe særlig, slik man må med designet fra cb76f55. Spesielt på mobil merkes det at det er veldig mye bortkastet plass, og det kreves mye scrolling for å se når alle lokalene er åpne, spesielt hvis du kun er interessert i å se alle åpningstidene til ett lokale.

Har et lite forslag til et design som løser det jeg nevnte. Sammenhengende dager med lik åpningstid kan grupperes sammen. Det hjelper for eksempel med å redusere støy for lokaler som kun er åpne i helger. Nåværende gjeldende åpningstid markeres med fet tekst.

image

@BragonSB

Copy link
Copy Markdown
Contributor Author

@robines Takk for feedback, enig i det du sier, var veldig usikker på den beste måten å vise informasjonen til brukeren, uten å overvelde dem, men backer designforslaget ditt.

Trenger endringene i #2096 for å fullføre, men kan fikse resten nå

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants